-
Notifications
You must be signed in to change notification settings - Fork 511
Matter Energy: Fix non- JSON serializable fields #2418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Matter Energy: Fix non- JSON serializable fields #2418
Conversation
Channel deleted. |
Test Results 71 files 455 suites 0s ⏱️ Results for commit 2450afd. ♻️ This comment has been updated with latest results. |
Minimum allowed coverage is Generated by 🐒 cobertura-action against 2450afd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spot test with VDA and I think this is good to go
abc9d86
to
2f36114
Compare
Tested with Battery Storage, Solar Power, and EVSE device types. Everything works fine except energyMeter is not populated because of read failures from the VDA for ElectricalEnergyMeasurement. Since this is a VDA issue, the behavior is the same on the production driver and the issue is unrelated to this change. |
Keys in datastore tables need to be strings in order to be JSON serializable. SUPPORTED_EVSE_MODES_MAP and SUPPORTED_DEVICE_ENERGY_MANAGEMENT_MODES_MAP need to be updated.
20b655b
to
2450afd
Compare
Keys in datastore tables need to be strings in order to be JSON serializable.
SUPPORTED_EVSE_MODES_MAP
andSUPPORTED_DEVICE_ENERGY_MANAGEMENT_MODES_MAP
need to be updated.